Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

prepare test transactions #777

Merged
merged 1 commit into from
Apr 1, 2025
Merged

prepare test transactions #777

merged 1 commit into from
Apr 1, 2025

Conversation

yuunlimm
Copy link
Contributor

Description

Checklist

  • If any existing pages were renamed or removed:
    • Were redirects added to next.config.mjs?
    • Did you update any relative links that pointed to the renamed / removed pages?
  • Do all Lints pass?
    • Have you ran pnpm fmt?
    • Have you ran pnpm lint?

@vercel Vercel
Copy link

vercel bot commented Jan 17, 2025 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
developer-docs-nextra 🛑 Canceled (Inspect) Apr 1, 2025 7:28pm

This was referenced Jan 17, 2025
Merged
Merged
@yuunlimm Graphite App
Copy link
Contributor Author

yuunlimm commented Jan 17, 2025 •
edited
Loading

This stack of pull requests is managed by Graphite. Learn more about stacking.

@yuunlimm yuunlimm mentioned this pull request Jan 17, 2025
Merged
4 tasks
@yuunlimm yuunlimm marked this pull request as ready for review January 22, 2025 21:04
@yuunlimm yuunlimm requested a review from a team as a code owner January 22, 2025 21:04
gregnazario
gregnazario reviewed Jan 22, 2025
View reviewed changes
Copy link
Collaborator

@gregnazario gregnazario left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This describes it, but how do you use it?

@yuunlimm yuunlimm force-pushed the 01-17-testing_processor branch from 3272d23 to 3661226 Compare January 23, 2025 18:16
@yuunlimm yuunlimm force-pushed the 01-17-prepare_test_transactions branch from dca6bb9 to fcc42bf Compare January 23, 2025 18:16
@yuunlimm yuunlimm force-pushed the 01-17-testing_processor branch from 3661226 to 3614d53 Compare January 23, 2025 18:22
@yuunlimm yuunlimm force-pushed the 01-17-prepare_test_transactions branch from fcc42bf to f1853fe Compare January 23, 2025 18:22
@yuunlimm yuunlimm force-pushed the 01-17-testing_processor branch from 3614d53 to a7aca8f Compare January 23, 2025 18:26
@yuunlimm yuunlimm force-pushed the 01-17-prepare_test_transactions branch from f1853fe to 05159b0 Compare January 23, 2025 18:26
@yuunlimm yuunlimm force-pushed the 01-17-testing_processor branch from a7aca8f to b77f255 Compare January 23, 2025 18:28
@yuunlimm yuunlimm force-pushed the 01-17-prepare_test_transactions branch from 05159b0 to 3c2f21e Compare January 23, 2025 18:28
@yuunlimm yuunlimm force-pushed the 01-17-testing_processor branch from b77f255 to e27f3c8 Compare January 23, 2025 18:30
@yuunlimm yuunlimm force-pushed the 01-17-prepare_test_transactions branch from 3c2f21e to 0e297bb Compare January 23, 2025 18:30
@yuunlimm yuunlimm force-pushed the 01-17-testing_processor branch from e27f3c8 to f9f765c Compare January 23, 2025 18:34
@yuunlimm yuunlimm force-pushed the 01-17-prepare_test_transactions branch from 0e297bb to 1b3497d Compare January 23, 2025 18:34
@yuunlimm yuunlimm force-pushed the 01-17-testing_processor branch from f9f765c to a287733 Compare January 23, 2025 19:47
@yuunlimm yuunlimm force-pushed the 01-17-prepare_test_transactions branch from 1b3497d to 6b7d434 Compare January 23, 2025 19:47
@yuunlimm yuunlimm force-pushed the 01-17-testing_processor branch from a287733 to 9992188 Compare January 23, 2025 21:17
@yuunlimm yuunlimm force-pushed the 01-17-prepare_test_transactions branch from 6b7d434 to 3c9a6ad Compare January 23, 2025 21:17
@yuunlimm yuunlimm force-pushed the 01-17-prepare_test_transactions branch from 4b5653a to 98af000 Compare February 8, 2025 01:00
@yuunlimm yuunlimm force-pushed the 01-17-testing_processor branch from 33b946d to a0d892b Compare February 8, 2025 01:03
@yuunlimm yuunlimm force-pushed the 01-17-prepare_test_transactions branch from 98af000 to db18fde Compare February 8, 2025 01:03
@yuunlimm yuunlimm force-pushed the 01-17-testing_processor branch from a0d892b to c438283 Compare February 8, 2025 01:30
@yuunlimm yuunlimm force-pushed the 01-17-prepare_test_transactions branch from db18fde to 8194f2c Compare February 8, 2025 01:31
@yuunlimm yuunlimm force-pushed the 01-17-testing_processor branch from c438283 to b1c98df Compare February 12, 2025 18:31
@yuunlimm yuunlimm force-pushed the 01-17-prepare_test_transactions branch from 8194f2c to dab56a1 Compare February 12, 2025 18:32
graphite-app[bot]
graphite-app bot reviewed Feb 12, 2025
View reviewed changes
apps/nextra/pages/en/build/indexer/indexer-sdk/advanced-tutorials/test-transactions.mdx
@@ -0,0 +1,18 @@
# Aptos Indexer Testing Framework Overview

The Aptos Indexer Testing Framework provides two ways to generate test transactions: **by Importing Transactions from Network** and **By writing a Move Scripts**. Both approaches are suited for specific scenarios based on your development and testing requirements, enabling you to test how your system handles various transaction types.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The capitalization and grammar in the two options should be consistent. Consider updating to "By Importing Transactions from Network" and "By Writing a Move Script" (note: "Move Scripts" should be singular since it refers to a single script).

Spotted by Graphite Reviewer

Is this helpful? React 👍 or 👎 to let us know.

@yuunlimm yuunlimm force-pushed the 01-17-testing_processor branch from b1c98df to 54969a0 Compare February 24, 2025 19:40
@yuunlimm yuunlimm force-pushed the 01-17-prepare_test_transactions branch from dab56a1 to c7d06b4 Compare February 24, 2025 19:40
@yuunlimm yuunlimm force-pushed the 01-17-testing_processor branch from 54969a0 to b11516a Compare April 1, 2025 18:36
@yuunlimm yuunlimm force-pushed the 01-17-prepare_test_transactions branch from c7d06b4 to 7d1305d Compare April 1, 2025 18:37
@yuunlimm yuunlimm force-pushed the 01-17-testing_processor branch 2 times, most recently from 855e32d to c09704d Compare April 1, 2025 18:54
@yuunlimm yuunlimm force-pushed the 01-17-prepare_test_transactions branch from 7d1305d to 6267987 Compare April 1, 2025 18:54
@yuunlimm yuunlimm changed the base branch from 01-17-testing_processor to graphite-base/777 April 1, 2025 19:16
@yuunlimm yuunlimm force-pushed the 01-17-prepare_test_transactions branch from 6267987 to ad4d900 Compare April 1, 2025 19:16
@yuunlimm yuunlimm force-pushed the graphite-base/777 branch from c09704d to 65c3d0c Compare April 1, 2025 19:16
@graphite-app graphite-app bot changed the base branch from graphite-base/777 to main April 1, 2025 19:17
@yuunlimm yuunlimm force-pushed the 01-17-prepare_test_transactions branch from ad4d900 to 12f9172 Compare April 1, 2025 19:17
@yuunlimm yuunlimm merged commit c13d837 into main Apr 1, 2025
4 of 5 checks passed
@yuunlimm Graphite App
Copy link
Contributor Author

yuunlimm commented Apr 1, 2025

Merge activity

  • Apr 1, 3:29 PM EDT: A user merged this pull request with Graphite.

@yuunlimm yuunlimm deleted the 01-17-prepare_test_transactions branch April 1, 2025 19:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gregnazario gregnazario gregnazario left review comments

@graphite-app graphite-app[bot] graphite-app[bot] left review comments

@rtso rtso rtso approved these changes

@hariria hariria Awaiting requested review from hariria

@igor-p igor-p Awaiting requested review from igor-p

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@yuunlimm @gregnazario @rtso